Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add basic overmap #15

Merged
merged 23 commits into from
Mar 5, 2024
Merged

Add basic overmap #15

merged 23 commits into from
Mar 5, 2024

Conversation

snipercup
Copy link
Collaborator

This requires #14 to work. I will publish this PR as a draft to make sure #14 gets merged first

This adds a basic overmap that allows you to change to a different tactical map:
image

In-game you can press m to open/close the map. To change a level, click one of the red tiles. The properties of the tile are shown on the right side. This is just a mockup and will change in the future.

If you click on a non-red tile, it will tell you to select a valid target:
image

Some things that can be improved on in the future that are not included in this PR:

  • The map seed changes when you switch level so you get a new overmap
  • Saving/loading any map data
  • Give the user better information about the location that's clicked
  • The overmap tiles are placeholders and will be replaced with something more recognizable
  • Improve performance while panning the overmap. Right now there's a small stutter when new chunks are added

@snipercup
Copy link
Collaborator Author

I didn't include the actual keybinding for m. I will include it in the next pr. So right now pressing m does nothing.

@snipercup snipercup mentioned this pull request Dec 5, 2023
@snipercup snipercup marked this pull request as ready for review March 5, 2024 15:32
@snipercup snipercup merged commit c83358d into Khaligufzel:main Mar 5, 2024
@snipercup snipercup deleted the overmap branch March 5, 2024 15:32
@snipercup snipercup restored the overmap branch March 5, 2024 15:32
@snipercup snipercup deleted the overmap branch March 5, 2024 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant